Skip to content

Autoroll from main to 27.lts - #11567

Merged
briantting merged 1 commit into
27.ltsfrom
autoroll-main-to-27.lts
Jul 28, 2026
Merged

Autoroll from main to 27.lts#11567
briantting merged 1 commit into
27.ltsfrom
autoroll-main-to-27.lts

Conversation

@cobalt-github-releaser-bot

Copy link
Copy Markdown
Collaborator

Automated cherry-pick roll to 27.lts.

Original pull requests:

…sink

Refer to original PR: #11549

Update accumulated_written_frames, last_playback_head_position, and
playback_head_position to use 64-bit integers in AudioTrackAudioSink.
At a 48kHz sampling rate, 32-bit signed integers overflow after
approximately 12.4 hours of continuous playback, which can cause
incorrect playback head reporting and audio sync issues.

Also update GetFramesDurationUs to accept int64_t to maintain type
consistency and prevent narrowing.

Note that this doesn't resolve all integer size issues across the audio
pipeline as the Java AudioTrackBridge is still tracking the playback
position using 32 bits integer, which will be resolve in b/539224216.

Bug: 539224216

(cherry picked from commit 97d9944)
@github-actions

Copy link
Copy Markdown
Contributor

🤖 Gemini Suggested Commit Message


android: Use 64-bit audio frame counts

Track audio frames using 64-bit integers instead of 32-bit integers in
the Android audio sink implementation. This prevents potential integer
overflows during long-running media playback, which can cause
inaccuracies in playback head reporting and synchronization.

This update ensures that accumulated frame counts and playback head
positions remain valid for extended durations. This is an automated
cherry-pick roll of PR #11549 to the 27.lts branch.

Issue: 11549

💡 Pro Tips for a Better Commit Message:

  1. Influence the Result: Want to change the output? You can write custom prompts or instructions directly in the Pull Request description. The model uses that text to generate the message.
  2. Re-run the Generator: Post a comment with: /generate-commit-message

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates several frame-tracking variables and function parameters in AudioTrackAudioSink from 32-bit integers to 64-bit integers (int64_t) to prevent potential integer overflow issues during long playback sessions. Additionally, the .github/AUTOROLL file was updated. There are no review comments, so I have no feedback to provide.

@briantting
briantting merged commit 89e50fb into 27.lts Jul 28, 2026
455 of 459 checks passed
@briantting
briantting deleted the autoroll-main-to-27.lts branch July 28, 2026 20:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants